Skip to content

chore(ci): narrow the deltalite wheel checkout - #127090

Merged
trunk-io[bot] merged 2 commits into
masterfrom
posthog/ci-deltalite-wheel-sparse-checkout
Nov 3, 2026
Merged

chore(ci): narrow the deltalite wheel checkout#127090
trunk-io[bot] merged 2 commits into
masterfrom
posthog/ci-deltalite-wheel-sparse-checkout

Conversation

@posthog

@posthog posthog Bot commented Nov 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Problem

Robot: DeltaLite wheel jobs check out repository files they do not use.

Why: Reduce unnecessary checkout work while preserving release artifacts.

Changes

  • The jobs check out rust/ and proto/ plus root .gitignore for Cargo packaging.
  • The wheel matrix and publishing gate stay unchanged.
  • No product behavior changes.

How did you test this code?

Test rationale: The release sdist build covers this regression.

  • The original sdist job failed because Cargo could not read root .gitignore.
  • A blobless depth-one clone reproduced and passed after the fix.
  • Sparse and full sdist archives match in contents.
  • Workflow lint and actionlint passed after the repair.
  • Preflight passed on the initial change.
  • The post-repair CI run is pending.

Release status

  • No feature flag controls this change

Docs update

  • No existing document covers this workflow.

🤖 Agent context

Autonomy: Fully autonomous

Agent: Codex, gpt-6.1-sol (initial change and repair validation); Codex, gpt-6-luna (CI repair)

Skills: /ci-shepherd, /authoring-ci-workflows, /verifying-changes, /running-ci-preflight, /writing-pr-descriptions

More context for AI agents
  • Session: Scout run.
  • Decisions: Keep the checkout narrow while providing Cargo the root ignore rules it needs to package the core crate.

Created with PostHog Desktop

@posthog
chore(ci): narrow the deltalite wheel checkout
e13f87b 
Generated-By: PostHog Desktop
Task-Id: 3543f873-94ad-474d-a186-adb62c554b6c
@trunk-io

trunk-io Bot commented Nov 2, 2026 •
edited
Loading

Copy link
Copy Markdown

😎 Merged successfully - details.

@github-actions

github-actions Bot commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

🤖 CI report

⚠️ Trunk lane — backend Python lane

This PR is assigned to the backend Python lane. It runs backend Python tests and may merge in parallel with PRs in other lanes.

posthog[bot]
posthog Bot commented Nov 2, 2026
View reviewed changes

@posthog posthog Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PostHog Review · Standard · reviewed e13f87b: no findings.

@posthog
posthog Bot marked this pull request as ready for review November 2, 2026 02:32
@parameterai

parameterai Bot commented Nov 2, 2026 •
edited
Loading

Copy link
Copy Markdown

Risk: No findings

The increment adds the root .gitignore to the sparse checkout used by the deltalite wheel-build jobs, restoring the ignore rules a full checkout would have so the CI sdist matches local builds. I verified it drops no file the deltalite build needs (the only tracked .gitignore matches under rust/ sit outside the deltalite crate) and changes no permissions, gating, or publish flow. No security issues introduced.

Sentinel reviewed d5d12f0 · Review settings

@pr-assigner-resolver-posthog
pr-assigner-resolver-posthog Bot requested a review from a team November 2, 2026 02:33
@posthog

posthog Bot commented Nov 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Robot: QA review complete.

Note

🤖 Automated comment by QA Swarm — not written by a human

Verdict: ✅ APPROVE (round 2 @ d5d12f0)

No open findings. The root ignore file fixes source packaging in a real blobless clone. The source archive matches a full checkout. The wheel matrix and publishing gates stay the same. All six wheel jobs and the source package passed. PostHog Review found no issues on this SHA. Stamphog requires a human reviewer for a bot-authored PR. Devex team review is requested.

Reviewer Assessment
Release review and repair validator (gpt-6.1-sol) Minimal repair accepted after reproduction and archive comparison.
Simplify (gpt-6-luna) No further changes.
Previous round

Round 1 @ e13f87b: review found no issues. CI then found a missing root ignore file in source packaging. Fixed in d5d12f0.

@hosthog

hosthog Bot commented Nov 2, 2026 •
edited
Loading

Copy link
Copy Markdown

HostHog preview — posthog-desktop-web

The previews for this PR have been torn down and no longer serve.

@posthog
chore(ci): include root gitignore for deltalite sdist
d5d12f0 
Robot: Cargo needs the root ignore rules to package the local core crate in the source distribution.

Generated-By: PostHog Desktop
Task-Id: 3543f873-94ad-474d-a186-adb62c554b6c
posthog[bot]
posthog Bot commented Nov 2, 2026
View reviewed changes

@posthog posthog Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PostHog Review · Standard · reviewed d5d12f0: no findings.

webjunkie
webjunkie approved these changes Nov 2, 2026
View reviewed changes

@webjunkie webjunkie left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 Agent review · steered by a person

Approving. The sparse checkout keeps the root .gitignore, so the earlier sdist failure stays fixed.

@robbie-c robbie-c mentioned this pull request Nov 2, 2026
Merged
@trunk-io
trunk-io Bot merged commit 7fb0edc into master Nov 3, 2026
244 checks passed
@trunk-io
trunk-io Bot deleted the posthog/ci-deltalite-wheel-sparse-checkout branch November 3, 2026 13:43
@deployment-status-posthog

deployment-status-posthog Bot commented Nov 3, 2026 •
edited
Loading

Copy link
Copy Markdown

Deploy status

Environment Status Deployed At Workflow
dev ✅ Deployed 2026-11-03 14:09 UTC Run
prod-us ✅ Deployed 2026-11-03 14:42 UTC Run
prod-eu ✅ Deployed 2026-11-03 14:35 UTC Run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@webjunkie webjunkie webjunkie approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@webjunkie